Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bazel] use patched python rules and build wheels cache #28

Merged
merged 1 commit into from
May 9, 2022

Conversation

timothytrippel
Copy link
Contributor

This repository is brought in as a Bazel dependency in the OpenTitan
project. In order to support offline (air-gapped) Bazel builds of
OpenTitan, all dependencies must also build in an air-gapped
environment. Since this repository make use of the Bazel Python rules,
Bazel will try to fetch and install Python packages using pip, will not
not cache them in Bazel's repository cache (as Bazel has no knowledge of
pip).

Therefore a similar approach to lowRISC/opentitan:#12091 must be taken:
that is, a custom repository rule must be added to cache Python wheels,
so that Bazel will not attempt to fetch them in the airgapped
environment if they are already present.

This partially addresses lowRISC/opentitan:#12538.

Signed-off-by: Timothy Trippel [email protected]

This repository is brought in as a Bazel dependency in the OpenTitan
project. In order to support offline (air-gapped) Bazel builds of
OpenTitan, all dependencies must also build in an air-gapped
environment. Since this repository make use of the Bazel Python rules,
Bazel will try to fetch and install Python packages using pip, will not
not cache them in Bazel's repository cache (as Bazel has no knowledge of
pip).

Therefore a similar approach to lowRISC/opentitan:#12091 must be taken:
that is, a custom repository rule must be added to cache Python wheels,
so that Bazel will not attempt to fetch them in the airgapped
environment if they are already present.

This partially addresses lowRISC/opentitan:#12538.

Signed-off-by: Timothy Trippel <[email protected]>
@mcy mcy merged commit 3c21d91 into lowRISC:master May 9, 2022
@timothytrippel timothytrippel deleted the use-patched-python-rules branch May 10, 2022 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants